Skip to content

Provide agent guidance on JSG_TRY vs. js.tryCatch - #7067

Merged
jasnell merged 1 commit into
mainfrom
jasnell/clarify-jsg-trycatch
Aug 25, 2026
Merged

Provide agent guidance on JSG_TRY vs. js.tryCatch#7067
jasnell merged 1 commit into
mainfrom
jasnell/clarify-jsg-trycatch

Conversation

@jasnell

@jasnell jasnell commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

The agents keep getting this wrong

@jasnell
jasnell requested review from guybedford and npaun August 19, 2026 15:28
@jasnell
jasnell requested review from a team as code owners August 19, 2026 15:28
@jasnell
jasnell enabled auto-merge August 19, 2026 15:28
Comment thread docs/reference/detail/legacy-module-registry.md
@ask-bonk

ask-bonk Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

I'm Bonk, and I've done a quick review of your PR.

Updates JSG exception-handling guidance and legacy module registry documentation.

  1. [P2] docs/reference/detail/legacy-module-registry.md:219 describes JSG_TRY/JSG_CATCH, but the resolver still uses js.tryCatch. Posted an inline suggestion.

github run

The agents keep getting this wrong
@jasnell
jasnell force-pushed the jasnell/clarify-jsg-trycatch branch from 05fe1a0 to c176f2b Compare August 25, 2026 13:39
@jasnell
jasnell merged commit ef56a46 into main Aug 25, 2026
20 checks passed
@jasnell
jasnell deleted the jasnell/clarify-jsg-trycatch branch August 25, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants